-
Notifications
You must be signed in to change notification settings - Fork 4
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: disableAutoSnapshot
not working as expected
#134
fix: disableAutoSnapshot
not working as expected
#134
Conversation
@jackleslie thanks for identifying the issue and making the fix! We'll take a look at this. |
Fix the linting error
Fix linting
@skitterm thanks for taking a look! I see that some checks are still failing, is there anything I need to do? |
@jackleslie you're good! I'll take it from here. Some checks require the PR to be from a non-fork branch to pass. I'll change this base branch to my (non-fork) branch, and after merging I'll make another PR from my branch to |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, thanks again @jackleslie!
accd2fa
into
chromaui:enable-manual-snapshots-without-automatic
Issue: #131
What Changed
From documentation:
I believe currently when
disableAutoSnapshot
is set totrue
we are not proceeding to theprepareArchives
task, so rather than disabling the automatic snapshot we are disabling all snapshotsHow to test
disableAutoSnapshot: true
globallycy.takeSnapshot()